Skip to content

Conversation

yeikel
Copy link
Collaborator

@yeikel yeikel commented Aug 22, 2025

It looks like Dependabot doesn’t handle this by default. With the current configuration, it still assigns the Spring dependencies to the default group instead of the dedicated Spring Boot group.

See #1271 for an example

What changes did you make? (Give an overview)

Is there anything you'd like reviewers to focus on?

This a follow up for #1181

How Has This Been Tested? (put an "x" (case-sensitive!) next to an item)

  • No need to

Checklist (put an "x" (case-sensitive!) next to all the items, otherwise the build will fail)

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (e.g. ENVIRONMENT VARIABLES)
  • My changes generate no new warnings (e.g. Sonar is happy)
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged

A picture of a cute animal (not mandatory but encouraged)

zorro-e1649086718471-1024x503

…group

It seems that dependabot does not do by default
@yeikel yeikel requested a review from a team as a code owner August 22, 2025 23:04
@kapybro kapybro bot added status/triage Issues pending maintainers triage scope/infra CI, CD, dev. env, etc. status/triage/manual Manual triage in progress status/triage/completed Automatic triage completed and removed status/triage Issues pending maintainers triage labels Aug 22, 2025
@yeikel
Copy link
Collaborator Author

yeikel commented Aug 22, 2025

@Haarolean FYI

@yeikel
Copy link
Collaborator Author

yeikel commented Aug 26, 2025

@germanosin if capacity permits. Thank you in advance

@Haarolean Haarolean changed the title Infra: Exclude Springboot dependencies from the other-dependencies group Infra: Exclude Springboot dependencies from the other-dependencies dependabot group Aug 26, 2025
@Haarolean Haarolean added type/chore Boring stuff, could be refactoring or tech debt and removed status/triage/manual Manual triage in progress labels Aug 26, 2025
@Haarolean Haarolean merged commit 4e7767b into kafbat:main Aug 26, 2025
49 checks passed
@yeikel yeikel deleted the patch-4 branch August 26, 2025 15:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
scope/infra CI, CD, dev. env, etc. status/triage/completed Automatic triage completed type/chore Boring stuff, could be refactoring or tech debt
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants